Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Check spelling with codespell #228

Merged
merged 3 commits into from
Feb 13, 2025
Merged

ci: Check spelling with codespell #228

merged 3 commits into from
Feb 13, 2025

Conversation

spetrosi
Copy link
Contributor

@spetrosi spetrosi commented Feb 7, 2025

  • You can ignore words inline by adding a comment like # codespell:ignore word.
  • You can ignore words by adding them to the .codespell_ignores file.
  • You can ignore files and directories by adding them with skip = to the .codespellrc file.

@spetrosi spetrosi self-assigned this Feb 7, 2025
@spetrosi spetrosi force-pushed the codespell-init branch 3 times, most recently from e4edf17 to a4465aa Compare February 7, 2025 14:55
Copy link
Collaborator

@natoscott natoscott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@spetrosi LGTM. Seems like we'll need to somehow filter the grafana.ini files though? (those are based on upstream provided configurations with minor template adaptations - previously we didn't attempt to tackle wording in that file)

@spetrosi spetrosi force-pushed the codespell-init branch 2 times, most recently from a4465aa to cce5fe3 Compare February 12, 2025 11:23
@spetrosi spetrosi changed the title ci: Replace woke with codespell ci: Check spelling with codespell Feb 12, 2025
* You can ignore words inline by adding a comment like `# codespell:ignore word`.
* You can ignore words by adding them to the `.codespell_ignores` file.
* You can ignore files and directories by adding them with `skip = ` to the `.codespellrc` file.

Signed-off-by: Sergei Petrosian <[email protected]>
@spetrosi spetrosi merged commit ff580f8 into main Feb 13, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants